Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOCS-758 Adds new commit scope rule degradation functionality for synchronous and group commit. #6044

Merged
merged 43 commits into from
Oct 8, 2024

Conversation

jpe442
Copy link
Contributor

@jpe442 jpe442 commented Sep 10, 2024

What Changed?

Adds new commit scope operation degradation functionality for synchronous commit and group commit. Affects both commit scope reference section and commit scopes feature section in PGD docs.

Tracked here in DOCS-758.

@jpe442 jpe442 changed the title Docs 758 Docs-758 Adds new commit scope rule degradation functionality for synchronous and group commit. Sep 10, 2024
@jpe442 jpe442 changed the title Docs-758 Adds new commit scope rule degradation functionality for synchronous and group commit. DOCS-758 Adds new commit scope rule degradation functionality for synchronous and group commit. Sep 10, 2024
@josh-heyer josh-heyer force-pushed the DOCS-978-Full-Branch-5.6 branch 2 times, most recently from 004687d to d9e248d Compare September 11, 2024 19:45
@jpe442 jpe442 marked this pull request as ready for review September 17, 2024 17:38
@jpe442 jpe442 requested a review from a team as a code owner September 17, 2024 17:38
@jpe442 jpe442 added the deploy Add this label to a PR and it will automatically be deployed to netlify label Sep 17, 2024
Copy link
Contributor

github-actions bot commented Sep 17, 2024

@github-actions github-actions bot temporarily deployed to pr-6044-docs-758 September 17, 2024 20:48 Inactive
@jpe442 jpe442 requested a review from djw-m September 17, 2024 20:50
@github-actions github-actions bot temporarily deployed to pr-6044-docs-758 September 17, 2024 20:53 Inactive
Copy link
Contributor

@djw-m djw-m left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This feels like it is missing a "Degrade options" entry under commit scopes which explains what the DEGRADE ON clause does and which commands support it.

Also the reordering should be applies to the commit scope rules section... whatever order that will be...

@josh-heyer
Copy link
Contributor

⚠Rebased! Pull latest before continuing work.

@github-actions github-actions bot temporarily deployed to pr-6044-docs-758 September 18, 2024 17:52 Inactive
@github-actions github-actions bot temporarily deployed to pr-6044-docs-758 September 18, 2024 18:32 Inactive
@github-actions github-actions bot temporarily deployed to pr-6044-docs-758 September 18, 2024 19:00 Inactive
@github-actions github-actions bot temporarily deployed to pr-6044-docs-758 September 18, 2024 19:18 Inactive
@github-actions github-actions bot temporarily deployed to pr-6044-docs-758 September 18, 2024 19:25 Inactive
@github-actions github-actions bot temporarily deployed to pr-6044-docs-758 September 19, 2024 18:29 Inactive
@github-actions github-actions bot temporarily deployed to pr-6044-docs-758 September 20, 2024 20:41 Inactive
@jpe442
Copy link
Contributor Author

jpe442 commented Sep 20, 2024

This feels like it is missing a "Degrade options" entry under commit scopes which explains what the DEGRADE ON clause does and which commands support it.

Also the reordering should be applies to the commit scope rules section... whatever order that will be...

Got it. Added Degrade options section and made reordering consistent throughout.

@github-actions github-actions bot temporarily deployed to pr-6044-docs-758 September 20, 2024 20:52 Inactive
@github-actions github-actions bot temporarily deployed to pr-6044-docs-758 September 24, 2024 19:11 Inactive
@github-actions github-actions bot temporarily deployed to pr-6044-docs-758 September 24, 2024 20:04 Inactive
jpe442 and others added 20 commits October 7, 2024 16:42
Signed-off-by: Dj Walker-Morgan <[email protected]>
Signed-off-by: Dj Walker-Morgan <[email protected]>
Signed-off-by: Dj Walker-Morgan <[email protected]>
Signed-off-by: Dj Walker-Morgan <[email protected]>
@jpe442 jpe442 requested a review from djw-m October 7, 2024 20:51
@github-actions github-actions bot temporarily deployed to pr-6044-docs-758 October 7, 2024 21:03 Inactive
Copy link
Contributor

@djw-m djw-m left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally LGTM.... lets get it merged and fix up in branch.

- '# Commit Scope kinds'
- synchronous_commit
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why synchronous_commit is being pushed ahead of group commit. Group commit, synchronous commit, camo and then lag control (by use/feature maturity?)

@jpe442 jpe442 merged commit 322319e into DOCS-978-Full-Branch-5.6 Oct 8, 2024
3 checks passed
@jpe442 jpe442 deleted the DOCS-758 branch October 8, 2024 14:19
djw-m added a commit that referenced this pull request Oct 15, 2024
…chronous and group commit. (#6044)

* First draft reference changes.

* Small changes.

* Small tweaks.

* More small changes in group and synchrnounous commit.

* Small fixes.

* More small tweaks.

* tried to fix link.

* Changing paths to absolute.

* Update product_docs/docs/pgd/5.6/commit-scopes/synchronous_commit.mdx

Co-authored-by: Dj Walker-Morgan <[email protected]>

* Update product_docs/docs/pgd/5.6/commit-scopes/index.mdx

Co-authored-by: Dj Walker-Morgan <[email protected]>

* Changed wording in Group commit.

* Changed order of commit scopes in Commit scope rules to feature synchronous commit first.

* changed overview to put synch commit before group commit.

* Removed some white space.

* Degrade options section.

* Added Degrade on section in Commit scopes page.

* Fixed some small things.

* Small word change.

* Move CAMO and adjust accordingly.

* Small changes.

* Moved degrading options to its own page.

* Addressed one of DJ's comments.

* Update product_docs/docs/pgd/5.6/commit-scopes/degrading.mdx

Co-authored-by: Dj Walker-Morgan <[email protected]>

* Update product_docs/docs/pgd/5.6/commit-scopes/degrading.mdx

Co-authored-by: Dj Walker-Morgan <[email protected]>

* Update product_docs/docs/pgd/5.6/commit-scopes/degrading.mdx

Co-authored-by: Dj Walker-Morgan <[email protected]>

* Update product_docs/docs/pgd/5.6/reference/commit-scopes.mdx

Co-authored-by: Dj Walker-Morgan <[email protected]>

* Update product_docs/docs/pgd/5.6/reference/commit-scopes.mdx

Co-authored-by: Dj Walker-Morgan <[email protected]>

* Moved behavior section up.

* Worked on wording given Petr's comments.

* More word changes.

* Add degrading bit to Overview.

* Added wrap-language support, wrapped blocks, added degrading to index

Signed-off-by: Dj Walker-Morgan <[email protected]>

* Remove console.log hangover

Signed-off-by: Dj Walker-Morgan <[email protected]>

* Improve feedback

Signed-off-by: Dj Walker-Morgan <[email protected]>

* Shorten tooltip

Signed-off-by: Dj Walker-Morgan <[email protected]>

* Update product_docs/docs/pgd/5.6/commit-scopes/degrading.mdx

Co-authored-by: Dj Walker-Morgan <[email protected]>

* Update product_docs/docs/pgd/5.6/commit-scopes/group-commit.mdx

Co-authored-by: Dj Walker-Morgan <[email protected]>

* Update product_docs/docs/pgd/5.6/commit-scopes/index.mdx

Co-authored-by: Dj Walker-Morgan <[email protected]>

* Update product_docs/docs/pgd/5.6/commit-scopes/synchronous_commit.mdx

Co-authored-by: Dj Walker-Morgan <[email protected]>

* Update product_docs/docs/pgd/5.6/commit-scopes/degrading.mdx

Co-authored-by: Dj Walker-Morgan <[email protected]>

* Disable startWrap for now

Signed-off-by: Dj Walker-Morgan <[email protected]>

* Fix link.

* Update product_docs/docs/pgd/5.6/commit-scopes/commit-scope-rules.mdx

Co-authored-by: Dj Walker-Morgan <[email protected]>

---------

Signed-off-by: Dj Walker-Morgan <[email protected]>
Co-authored-by: Dj Walker-Morgan <[email protected]>
Co-authored-by: Dj Walker-Morgan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy Add this label to a PR and it will automatically be deployed to netlify
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants